Finish ystack public rebrand with pstack compatibility - #5
Conversation
Promote ystack as the install target and primary skill entry, add setup-ystack and ystack-named model overrides, retain pstack aliases and legacy config fallbacks, refresh user-facing guides, and add a CI branding guard. Co-authored-by: ChatGPT <noreply@openai.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 051f6f7a7c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | `bug_impl` | evidence-backed bug, performance, and reliability fixes | | ||
| | `judgment` | architecture, synthesis, and prose | | ||
| | `critic` | independent candidates and adversarial review | | ||
| 1. Read `../ystack/SKILL.md` in full before taking any action. |
There was a problem hiding this comment.
Preserve a fallback for standalone legacy installs
When an existing user selectively installs or updates only the legacy pstack skill, this required read targets a newly introduced sibling that is absent; before this commit, the router and its runtime assets lived entirely under skills/pstack. The alias therefore cannot execute /pstack unless the user reinstalls the complete pack, undermining the stated compatibility for existing installations. Keep the previous router as a fallback or make the alias self-contained.
Useful? React with 👍 / 👎.
What changed
ystackas the public repository, install target, and primary skill entry;/ystackand/setup-ystackskills;/pstackand/setup-pstackas explicit compatibility aliases;ystack-models.*override files while continuing to read legacypstack-models.*andcodex-pstack-models.mdpaths;Go7hic/pstackrepository slug and verifies the public/legacy entry files;Naming boundary
pstack,setup-pstack, andpstack-models.*skills/pstack/Keeping the internal runtime path avoids a large breaking migration while making every new user-facing surface consistently ystack.
User impact
New installs use:
npx skills add Go7hic/ystack -g -s '*' -yPrimary commands are:
Existing prompts and model configuration remain usable through compatibility aliases and adapter fallbacks.
Validation
The repository workflow checks:
Review focus
skills/pstack/as the shared internal runtime path is the right low-risk migration boundary;